Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add constructor args example #2

Merged
merged 2 commits into from
Nov 13, 2024
Merged

feat: add constructor args example #2

merged 2 commits into from
Nov 13, 2024

Conversation

chadoh
Copy link
Contributor

@chadoh chadoh commented Oct 28, 2024

Copying in the example from https://github.com/stellar/stellar-cli/pull/1574/files#diff-0d743a0fa81fee10cf91dbf7fb7480daa248d14468d1cecfa24577ef00597f5b

Also:

  • add a test for it
  • upgrade to p22 for whole workspace
  • gitignore test_snapshots

We can circle back and use this repo as a submodule in the CLI once this is merged, as we do already in js-stellar-sdk.

chadoh added a commit to AhaLabs/js-stellar-sdk that referenced this pull request Oct 28, 2024
@chadoh chadoh force-pushed the feat/constructor_args branch from 5e5471e to 06ae7be Compare October 28, 2024 19:46
Copy link
Member

@willemneal willemneal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one issue

@chadoh chadoh force-pushed the feat/constructor_args branch 2 times, most recently from 83b68a6 to c417a31 Compare November 1, 2024 18:51
chadoh added a commit to AhaLabs/js-stellar-sdk that referenced this pull request Nov 8, 2024
Copying in the example from https://github.com/stellar/stellar-cli/pull/1574/files#diff-0d743a0fa81fee10cf91dbf7fb7480daa248d14468d1cecfa24577ef00597f5b

Also:

- add a test for it
- upgrade to p22 for whole workspace
- gitignore test_snapshots

We can circle back and use this repo as a submodule in the CLI once this
is merged, as we do already in js-stellar-sdk.
@chadoh chadoh force-pushed the feat/constructor_args branch from c417a31 to d3b5f33 Compare November 12, 2024 19:17
@chadoh chadoh requested a review from willemneal November 12, 2024 19:18
Copy link

@Shaptic Shaptic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a comment about the "learnability" aspect of the actual contract

Cargo.toml Outdated Show resolved Hide resolved
constructor-args/Cargo.toml Outdated Show resolved Hide resolved
constructor-args/src/lib.rs Outdated Show resolved Hide resolved
rm constructor-args, it doesn't add much
@chadoh chadoh enabled auto-merge (squash) November 13, 2024 19:15
@chadoh chadoh merged commit 50aca41 into main Nov 13, 2024
@chadoh chadoh deleted the feat/constructor_args branch November 13, 2024 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants